Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

gpm-backlight: unused function 'gpm_common_sum_scale' #366

Merged
merged 1 commit into from
Mar 11, 2022

Conversation

rbuj
Copy link
Contributor

@rbuj rbuj commented Oct 14, 2021

It removes the build warning below:

gpm-backlight.c:263:1: warning: unused function 'gpm_common_sum_scale' [-Wunused-function]

configure summary:

                    MATE Power Manager 1.26.0
                  =============================

        prefix:                    /usr
        datadir:                   ${datarootdir}
        compiler:                  clang
        cflags:                    -g -O0 -Wconversion -Wunused-macros -Wunused-parameter
        cwarnings:                 -Wall -Wmissing-prototypes -Wbad-function-cast -Wcast-align -Wextra -Wno-unused-parameter -Wformat-nonliteral -Wmissing-declarations -Wmissing-field-initializers -Wnested-externs -Wpointer-arith -Wredundant-decls -Wshadow -Wstrict-prototypes -Werror=format-security
        libsecret support:         yes
        gnome-keyring support:     no
        Building extra applets:    yes
        Self test support:         no
        dbus-1 services dir:       ${datarootdir}/dbus-1/services

Copy link
Member

@zhuyaliang zhuyaliang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Warning removed

Copy link
Member

@raveit65 raveit65 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@raveit65 raveit65 merged commit 2a42090 into master Mar 11, 2022
@raveit65 raveit65 deleted the gpm_common_sum_scale branch March 11, 2022 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants